Skip to content
This repository has been archived by the owner on Nov 19, 2018. It is now read-only.

Update AWS PHP SDK to v3 #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jaredcassidy
Copy link

No description provided.

Copy link

@akazakou akazakou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need fix my comments for both files, because right now it doesn't work

return new S3Client([
'region' => $s3Region,
'version' => '2006-03-01',
'signature_version' => $s3Signature_version,

This comment was marked as spam.

@@ -68,6 +72,10 @@
// and other POST requests (all requests are sent to the same endpoint in this example).
// This condition is not needed if you don't require a callback on upload success.
if (isset($_REQUEST["success"])) {
//If Region is set in uploadSuccess params then override the default us-east-1
$s3Region = (isset($_REQUEST["region"]) ? $_REQUEST["region"] : 'us-east-1');

This comment was marked as spam.

// The following variables are used when validating the policy document
// sent by the uploader.
$expectedBucketName = $_ENV['S3_BUCKET_NAME'];
$expectedHostName = $_ENV['S3_HOST_NAME']; // v4-only

This comment was marked as spam.

return new S3Client([
'region' => $s3Region,
'version' => '2006-03-01',
'signature_version' => $s3Signature_version,

This comment was marked as spam.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants